Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(operator): fix cpu stress and memory stress builder #405

Merged
merged 4 commits into from
Jun 8, 2023
Merged

fix(operator): fix cpu stress and memory stress builder #405

merged 4 commits into from
Jun 8, 2023

Conversation

mlycore
Copy link
Contributor

@mlycore mlycore commented Jun 8, 2023

Type of change:

  • Bugfix
  • New feature provided
  • Improve performance
  • Backport patches

What this PR does / why we need it:

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?

mlycore added 3 commits June 8, 2023 15:12
@mlycore
feat: introduce StressChaos in builder
1ef79ad 
Signed-off-by: mlycore <maxwell92@126.com>
@mlycore
fix: fix stresschaos get
c44b6e7 
Signed-off-by: mlycore <maxwell92@126.com>
@mlycore
fix: fix memory chaos convert
d186589 
Signed-off-by: mlycore <maxwell92@126.com>
@mlycore mlycore added this to the 0.3.0 milestone Jun 8, 2023
@github-actions github-actions bot added the operator solutions of operator label Jun 8, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jun 8, 2023 •
edited
Loading

Codecov Report

Merging #405 (eb58e3a) into main (3604a24) will decrease coverage by 1.33%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #405      +/-   ##
==========================================
- Coverage   39.71%   38.38%   -1.33%     
==========================================
  Files          51       52       +1     
  Lines        6083     6335     +252     
==========================================
+ Hits         2416     2432      +16     
- Misses       3448     3674     +226     
- Partials      219      229      +10
Impacted Files Coverage Δ
...ator/cmd/shardingsphere-operator/manager/option.go 49.53% <0.00%> (ø)
...phere-operator/pkg/controllers/chaos_controller.go 0.00% <0.00%> (ø)

... and 4 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mlycore
fix: fix lint and e2e test
eb58e3a 
Signed-off-by: mlycore <maxwell92@126.com>
@mlycore mlycore added in: chaos modules of shardingsphere chaos type: fixed labels Jun 8, 2023
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 8, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@strongduanmu strongduanmu merged commit 93ddb65 into apache:main Jun 8, 2023
@mlycore mlycore deleted the update-stress branch June 8, 2023 08:29
@mlycore
Copy link
Contributor Author

mlycore commented Jul 11, 2023

ref #290

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@strongduanmu strongduanmu strongduanmu approved these changes

Assignees
No one assigned
Labels
in: chaos modules of shardingsphere chaos operator solutions of operator type: fixed
Projects
None yet
Milestone
0.3.0
Development

Successfully merging this pull request may close these issues.
3 participants
@mlycore @codecov-commenter @strongduanmu